Skip to content

fix(wrapper): correct disable guidance - #1607

Merged
John-David Dalton (jdalton) merged 1 commit into
v1.xfrom
fix/wrapper-off-guidance-v1x
Nov 11, 2026
Merged

John-David Dalton (jdalton) merged 1 commit into
v1.xfrom
fix/wrapper-off-guidance-v1x

Conversation

@jdalton

@jdalton John-David Dalton (jdalton) commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Give users the working socket wrapper off command after enabling the shell wrapper.

A CLI regression test checks the guidance and confirms that disabling the wrapper removes both aliases.

Automated review notes

[!NOTE]
Low Risk
User-facing copy and test-only changes; no change to wrapper enable/disable behavior beyond fixing incorrect instructions.

Overview
Corrects the post-enable success message so it tells users to run socket wrapper off instead of the non-existent socket wrapper --disable, aligning guidance with the documented on / off subcommands.

Adds an integration test that enables the wrapper in a temporary home, asserts the printed disable hint and shell aliases, then runs wrapper off and confirms both npm and npx aliases are removed from the profile.

Reviewed by Cursor Bugbot for commit 49a9a88. Configure here.

@jdalton
John-David Dalton (jdalton) merged commit 5baf3f0 into v1.x Nov 11, 2026
17 checks passed
@jdalton
John-David Dalton (jdalton) deleted the fix/wrapper-off-guidance-v1x branch November 11, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants